Skip to content

[Skip Issue] Fix a "double decref" in termios_tcgetattr()#10194

Merged
serhiy-storchaka merged 1 commit into
python:masterfrom
ZackerySpytz:termios_tcgetattr-ref-count-bug
Oct 29, 2018
Merged

[Skip Issue] Fix a "double decref" in termios_tcgetattr()#10194
serhiy-storchaka merged 1 commit into
python:masterfrom
ZackerySpytz:termios_tcgetattr-ref-count-bug

Conversation

@ZackerySpytz

Copy link
Copy Markdown
Contributor

"cc" will be decrefed twice if an error occurs in a PyLong_FromLong()
call towards the end of termios_tcgetattr().

@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @ZackerySpytz for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 2.7, 3.6, 3.7.
🐍🍒⛏🤖

@bedevere-bot

Copy link
Copy Markdown

GH-10210 is a backport of this pull request to the 3.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 29, 2018
(cherry picked from commit 53835e9)

Co-authored-by: Zackery Spytz <zspytz@gmail.com>
@bedevere-bot

Copy link
Copy Markdown

GH-10211 is a backport of this pull request to the 3.6 branch.

@miss-islington

Copy link
Copy Markdown
Contributor

Sorry, @ZackerySpytz and @serhiy-storchaka, I could not cleanly backport this to 2.7 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 53835e92d315340444e3dd083b3f69a590b00e07 2.7

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 29, 2018
(cherry picked from commit 53835e9)

Co-authored-by: Zackery Spytz <zspytz@gmail.com>
miss-islington added a commit that referenced this pull request Oct 29, 2018
(cherry picked from commit 53835e9)

Co-authored-by: Zackery Spytz <zspytz@gmail.com>
miss-islington added a commit that referenced this pull request Oct 29, 2018
(cherry picked from commit 53835e9)

Co-authored-by: Zackery Spytz <zspytz@gmail.com>
@bedevere-bot

Copy link
Copy Markdown

Hi! The buildbot AMD64 Debian PGO 3.x has failed when building commit 53835e9.

You can take a look here:

https://buildbot.python.org/all/#builders/47/builds/1755

@serhiy-storchaka

Copy link
Copy Markdown
Member

Do you mind to create a backport for 2.7 @ZackerySpytz?

ZackerySpytz added a commit to ZackerySpytz/cpython that referenced this pull request Oct 29, 2018
@bedevere-bot

Copy link
Copy Markdown

GH-10218 is a backport of this pull request to the 2.7 branch.

@ZackerySpytz

Copy link
Copy Markdown
Contributor Author

Serhiy, I've created a backport for 2.7.

@ZackerySpytz ZackerySpytz deleted the termios_tcgetattr-ref-count-bug branch October 29, 2018 15:38
@serhiy-storchaka serhiy-storchaka removed their assignment Oct 29, 2018
serhiy-storchaka pushed a commit that referenced this pull request Oct 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip issue skip news type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants